Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix HC3 compatibility further #38931

Merged
merged 2 commits into from
Aug 17, 2020
Merged

Conversation

pbalogh77
Copy link
Contributor

Further fixes for HC3 compatibility.

Proposed change

Sometimes the new Fibaro HC3 misses important fields for scenes. Making sure such event does not result in a crash of the integration.

Type of change

  • Dependency upgrade

  • Bugfix (non-breaking change which fixes an issue)

  • New integration (thank you!)

  • New feature (which adds functionality to an existing integration)

  • Breaking change (fix/feature causing existing functionality to break)

  • Code quality improvements to existing code or addition of tests

  • This PR fixes or closes issue: fixes

  • This PR is related to issue: Fibaro Integration with HC3 #38373

  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist
  • The code has been formatted using Black (black --fast homeassistant tests)
  • Tests have been added to verify that the new code works.

To help with the load of incoming pull requests:

Further fixes for HC3 compatibility.
@MartinHjelmare MartinHjelmare added this to Needs review in Dev Aug 16, 2020
@MartinHjelmare MartinHjelmare changed the title Further fixes for HC3 compatibility Fix HC3 compatibility further Aug 16, 2020
Dev automation moved this from Needs review to Review in progress Aug 16, 2020
Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
Copy link
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Dev automation moved this from Review in progress to Reviewer approved Aug 17, 2020
@MartinHjelmare MartinHjelmare merged commit 5a4de04 into home-assistant:dev Aug 17, 2020
Dev automation moved this from Reviewer approved to Done Aug 17, 2020
@MartinHjelmare MartinHjelmare added this to the 0.114.2 milestone Aug 17, 2020
@pbalogh77 pbalogh77 deleted the hc3-fixes branch August 17, 2020 06:33
balloob pushed a commit that referenced this pull request Aug 17, 2020
* Update __init__.py

Further fixes for HC3 compatibility.

* Update homeassistant/components/fibaro/__init__.py

Co-authored-by: Martin Hjelmare <marhje52@gmail.com>

Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
@balloob balloob mentioned this pull request Aug 17, 2020
@kokosmith
Copy link

Hello @pbalogh77. I have changed from Fibaro HC2 to HC3 and the Fibaro HA integration does not work by providing the local IP of the fibaro. Could you pls let me know if the HC3 is still supported or I may do something wrong? Thanks

@MartinHjelmare
Copy link
Member

Please open an issue instead. Thanks!

@home-assistant home-assistant locked as resolved and limited conversation to collaborators Jul 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Dev
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants